-
Notifications
You must be signed in to change notification settings - Fork 735
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds CODEOWNERS
file to streamline notifications
#807
Conversation
Looks like a cool feature to me! I would personally do not include teams as there might be a lot of changes within the people. But that can be up for discussion |
@pascal-roth Thanks! Please add any section of the code for which you would like to be a reviewer for as suggestions |
CODEOWNERS
file with a few examplesCODEOWNERS
file with current developer team
@jsmith-bdai I took a stab at having a more thorough listing in there. Still a pretty tentative list. We can work on improving it in our discussions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added suggestions for some areas of interest for me.
Co-authored-by: James Smith <[email protected]> Co-authored-by: jtigue-bdai <[email protected]> Signed-off-by: Mayank Mittal <[email protected]>
CODEOWNERS
file with current developer teamCODEOWNERS
file to streamline notifications
Description
This PR adds in a CODEOWNERS file so that we can better handle the review request process.
NOTE: This version is not finalized, please add suggestions to other areas on this PR. We can constantly add more as well, I just wanted to get an initial version up and filled in the things I knew off the top of my head 😄
Type of change
Checklist
pre-commit
checks with./isaaclab.sh --format
config/extension.toml
fileCONTRIBUTORS.md
or my name already exists there